New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Cleanup error logging #79

Merged

timja merged 1 commit into main from finish-errors

Aug 11, 2022

Owner

timja commented Aug 11, 2022 •

edited

Loading

Remove console.error
Don't fail action if reactions are broken

Logging done at the command level so we don't need to pass ID and user down to github level

relates to #59


          Cleanup error logging

5c5096f

timja added the enhancement label

timja merged commit f938ac4 into main

timja deleted the finish-errors branch

August 11, 2022 07:55

timja mentioned this pull request

Add thumbs up reaction #78

Merged

6 tasks

Contributor

jetersen commented Aug 11, 2022

Thanks looks like a healthy cleanup!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels